Skip to content

Conversation

jbrockmendel
Copy link
Member

its not used anywhere, apparently legacy leftover

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@gfyoung gfyoung added Clean Internals Related to non-user accessible pandas implementation labels Jan 25, 2020
@gfyoung gfyoung added this to the 1.1 milestone Jan 25, 2020
@gfyoung
Copy link
Member

gfyoung commented Jan 25, 2020

https://github.com/pandas-dev/pandas/search?q=_subtyp&unscoped_q=_subtyp

@jbrockmendel : Do you intend to remove any remaining ones after this (as a follow-up)? Or could we tackle all of them in this PR?

@jreback jreback merged commit 2038d7a into pandas-dev:master Jan 25, 2020
@jreback
Copy link
Contributor

jreback commented Jan 25, 2020

sure

@jbrockmendel
Copy link
Member Author

Do you intend to remove any remaining ones after this

You mean legacy methods? Nothing actively planned, just as I come across them. Or are you suggesting _subtyp should be removed from _internal_names?

@jbrockmendel jbrockmendel deleted the cln-subtyp branch January 25, 2020 16:04
@jreback
Copy link
Contributor

jreback commented Jan 25, 2020

Do you intend to remove any remaining ones after this

You mean legacy methods? Nothing actively planned, just as I come across them. Or are you suggesting _subtyp should be removed from _internal_names?

this actually migtht be doable

keechongtan added a commit to keechongtan/pandas that referenced this pull request Jan 27, 2020
…ndexing-1row-df

* upstream/master: (194 commits)
  DOC Remove Python 2 specific comments from documentation (pandas-dev#31198)
  Follow up PR: pandas-dev#28097 Simplify branch statement (pandas-dev#29243)
  BUG: DatetimeIndex.snap incorrectly setting freq (pandas-dev#31188)
  Move DataFrame.info() to live with similar functions (pandas-dev#31317)
  ENH: accept a dictionary in plot colors (pandas-dev#31071)
  PERF: add shortcut to Timestamp constructor (pandas-dev#30676)
  CLN/MAINT: Clean and annotate stata reader and writers (pandas-dev#31072)
  REF: define _get_slice_axis in correct classes (pandas-dev#31304)
  BUG: DataFrame.floordiv(ser, axis=0) not matching column-wise bheavior (pandas-dev#31271)
  PERF: optimize is_scalar, is_iterator (pandas-dev#31294)
  BUG: Series rolling count ignores min_periods (pandas-dev#30923)
  xfail sparse warning; closes pandas-dev#31310 (pandas-dev#31311)
  REF: DatetimeIndex.get_value wrap DTI.get_loc (pandas-dev#31314)
  CLN: internals.managers (pandas-dev#31316)
  PERF: avoid copies if possible in fill_binop (pandas-dev#31300)
  Add test for multiindex json (pandas-dev#31307)
  BUG: passing TDA and wrong freq to TimedeltaIndex (pandas-dev#31268)
  BUG: inconsistency between PeriodIndex.get_value vs get_loc (pandas-dev#31172)
  CLN: remove _set_subtyp (pandas-dev#31301)
  CI: Updated version of macos image (pandas-dev#31292)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean Internals Related to non-user accessible pandas implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants